Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

HBASE-22272 Fixed Checkstyle errors in hbase-backup #165

Merged
merged 1 commit into from
Apr 24, 2019
Merged

HBASE-22272 Fixed Checkstyle errors in hbase-backup #165

merged 1 commit into from
Apr 24, 2019

Conversation

HorizonNet
Copy link
Contributor

No description provided.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 45 Docker mode activated.
_ Prechecks _
+1 hbaseanti 0 Patch does not have any anti-patterns.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1 mvninstall 248 master passed
+1 compile 24 master passed
+1 checkstyle 15 master passed
+1 shadedjars 263 branch has no errors when building our shaded downstream artifacts.
+1 findbugs 39 master passed
+1 javadoc 18 master passed
_ Patch Compile Tests _
+1 mvninstall 243 the patch passed
+1 compile 23 the patch passed
+1 javac 23 the patch passed
+1 checkstyle 14 hbase-backup: The patch generated 0 new + 0 unchanged - 4 fixed = 0 total (was 4)
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 264 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 501 Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1 findbugs 47 the patch passed
+1 javadoc 16 the patch passed
_ Other Tests _
+1 unit 989 hbase-backup in the patch passed.
+1 asflicense 15 The patch does not generate ASF License warnings.
2838
Subsystem Report/Notes
Docker Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-165/1/artifact/out/Dockerfile
GITHUB PR #165
Optional Tests dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 370d779a025d 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build tool maven
Personality /testptch/patchprocess/precommit/personality/provided.sh
git revision master / 268bcce
maven version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java 1.8.0_181
findbugs v3.1.11
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-165/1/testReport/
Max. process+thread count 4465 (vs. ulimit of 10000)
modules C: hbase-backup U: hbase-backup
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-165/1/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

This was referenced Apr 19, 2019
Merged
Merged
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 287 Docker mode activated.
_ Prechecks _
+1 hbaseanti 0 Patch does not have any anti-patterns.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1 mvninstall 323 master passed
+1 compile 30 master passed
+1 checkstyle 17 master passed
+1 shadedjars 329 branch has no errors when building our shaded downstream artifacts.
+1 findbugs 51 master passed
+1 javadoc 19 master passed
_ Patch Compile Tests _
+1 mvninstall 295 the patch passed
+1 compile 28 the patch passed
+1 javac 28 the patch passed
+1 checkstyle 16 hbase-backup: The patch generated 0 new + 0 unchanged - 4 fixed = 0 total (was 4)
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 335 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 620 Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1 findbugs 58 the patch passed
+1 javadoc 19 the patch passed
_ Other Tests _
+1 unit 1079 hbase-backup in the patch passed.
+1 asflicense 15 The patch does not generate ASF License warnings.
3592
Subsystem Report/Notes
Docker Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-165/2/artifact/out/Dockerfile
GITHUB PR #165
Optional Tests dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 4e962a9820a4 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build tool maven
Personality /testptch/patchprocess/precommit/personality/provided.sh
git revision master / f30d6c9
maven version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java 1.8.0_181
findbugs v3.1.11
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-165/2/testReport/
Max. process+thread count 4542 (vs. ulimit of 10000)
modules C: hbase-backup U: hbase-backup
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-165/2/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

This was referenced Apr 20, 2019
Closed
Merged
Merged
@Apache-HBase Apache-HBase mentioned this pull request Apr 23, 2019
Merged
@HorizonNet HorizonNet merged commit d3bf9c0 into apache:master Apr 24, 2019
@HorizonNet HorizonNet deleted the HBASE-22272 branch April 24, 2019 11:41
This was referenced Apr 24, 2019
Merged
Merged
Merged
Merged
Merged
Closed
Merged
Merged
This was referenced May 2, 2019
Merged
Merged
This was referenced May 9, 2019
Closed
Closed
Merged
This was referenced May 21, 2019
Merged
Merged
This was referenced May 28, 2019
Merged
Merged
This was referenced Jun 22, 2023
Merged
Closed
Merged
Merged
Merged
Merged
This was referenced Jul 3, 2023
Open
Open
Open
Merged
Merged
This was referenced Jul 14, 2023
Merged
Merged
Merged
This was referenced Jul 23, 2023
Merged
Merged
Merged
Merged
Merged
Merged
Merged
This was referenced Jul 31, 2023
Merged
Merged
Merged
Closed
Closed
This was referenced Aug 8, 2023
Merged
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saintstack saintstack saintstack approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@HorizonNet @Apache-HBase @saintstack